Skillet: fix sparse checkout path typing in pre-activation skills checkout#40684
Merged
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix skills sparse-checkout step in Skillet
Skillet: fix sparse checkout path typing in pre-activation skills checkout
Jun 21, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Skillet workflow’s pre-activation sparse checkout configuration to avoid git sparse-checkout failures in cone mode by replacing workflow file entries under .github/workflows/ with the directory path, and then syncing the compiled workflow output.
Changes:
- Adjusted Skillet’s pre-activation
actions/checkoutsparse-checkoutlist to include.github/workflows(directory) instead of specific workflow files. - Recompiled
skillet.lock.ymlso the runtime workflow YAML reflects the updated sparse-checkout set.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/skillet.md | Updates pre-activation sparse-checkout paths used before skill matching/agent startup. |
| .github/workflows/skillet.lock.yml | Syncs the compiled workflow YAML with the updated sparse-checkout configuration. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 1
Comment on lines
45
to
49
| sparse-checkout: | | ||
| .github/skills | ||
| .github/workflows/agentic_commands.yml | ||
| .github/workflows/agentic-maintenance.yml | ||
| .github/workflows | ||
| actions/setup/js/slash_command_matcher.cjs | ||
| persist-credentials: false |
pelikhan
reviewed
Jun 21, 2026
| .github/workflows/agentic_commands.yml | ||
| .github/workflows/agentic-maintenance.yml | ||
| .github/workflows | ||
| actions/setup/js/slash_command_matcher.cjs |
Contributor
Author
There was a problem hiding this comment.
Updated this in acd8fb5: the sparse-checkout entry now uses the directory path actions/setup/js (cone-mode compatible), and the compiled skillet.lock.yml is synced.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
github-actions Bot
added a commit
that referenced
this pull request
Jun 22, 2026
Weekly update covering the week of June 15–22, 2026: - Compiler +320% performance regression fix (#40662) - New deferinloop Go linter (#40679) - gh-aw-detection rollout to 50% of workflows (#40698) - JSON-RPC error handling fix (#40715) - Skillet sparse checkout path fix (#40684) - FNV-1a heredoc delimiter generation (#40696) - Agent of the Week: delight ✨ Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skilletworkflow_dispatchruns were failing inCheckout skills directorybecause sparse-checkout (cone mode) included file paths under.github/workflowsas directory-style entries, causinggitexit 128 before activation. This change normalizes checkout scope to directory paths so pre-activation can proceed to skill matching/agent startup.Problem scope
.github/skills(directory).github/workflows/agentic_commands.yml(file).github/workflows/agentic-maintenance.yml(file)actions/setup/js/slash_command_matcher.cjs(file)fatal: '... is not a directory'.Workflow update
/.github/workflows/skillet.mdCheckout skills directoryto include only directory entries:.github/skills.github/workflowsactions/setup/jsagentic_commands.yml,agentic-maintenance.yml, andslash_command_matcher.cjs.Compiled workflow sync
/.github/workflows/skillet.lock.ymlso runtime YAML reflects the corrected sparse-checkout set.pr-sous-chef: update branch requested for workflow triage
run: 27916230804